Skip to content

ci: Resolve CI failures#8347

Merged
pskiran1 merged 3 commits into
mainfrom
spolisetty_fix_ci_failures
Aug 15, 2025
Merged

ci: Resolve CI failures#8347
pskiran1 merged 3 commits into
mainfrom
spolisetty_fix_ci_failures

Conversation

@pskiran1

@pskiran1 pskiran1 commented Aug 14, 2025

Copy link
Copy Markdown
Member

What does the PR do?

As part of the changes in #8336, an empty shm_key is no longer allowed. The test case has been updated accordingly.

Checklist

  • PR title reflects the change and is of format <commit_type>: <Title>
  • Changes are described in the pull request.
  • Related issues are referenced.
  • Populated github labels field
  • Added test plan and verified test passes.
  • Verified that the PR passes existing CI.
  • Verified copyright is correct on all changed files.
  • Added succinct git squash message before merging ref.
  • All template sections are filled out.
  • Optional: Additional screenshots for behavior/output changes with before/after.

Commit Type:

Check the conventional commit type
box here and add the label to the github PR.

  • build
  • ci
  • docs
  • feat
  • fix
  • perf
  • refactor
  • revert
  • style
  • test

Related PRs:

Where should the reviewer start?

Test plan:

  • CI Pipeline ID: 33262629

Caveats:

Background

Related Issues: (use one of the action keywords Closes / Fixes / Resolves / Relates to)

  • closes GitHub issue: #xxx

@pskiran1 pskiran1 added the PR: ci Changes to our CI configuration files and scripts label Aug 14, 2025
@pskiran1
pskiran1 marked this pull request as ready for review August 14, 2025 08:28
@pskiran1
pskiran1 requested review from mc-nv and yinggeh August 14, 2025 14:09
@mc-nv
mc-nv self-requested a review August 15, 2025 03:14
@pskiran1
pskiran1 merged commit d3817a1 into main Aug 15, 2025
3 checks passed
@pskiran1
pskiran1 deleted the spolisetty_fix_ci_failures branch August 15, 2025 03:16
pskiran1 added a commit that referenced this pull request Aug 15, 2025
@pskiran1 pskiran1 mentioned this pull request Aug 15, 2025
20 tasks
mc-nv pushed a commit that referenced this pull request Aug 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: ci Changes to our CI configuration files and scripts

Development

Successfully merging this pull request may close these issues.

2 participants